04. 管理活跃 PR
项目维护者可能不会立即决定接受你的更改。他们可能会要求你对代码做一些额外更改,才会接受你的请求并合并你的更改。他们很可能会通过 Pull Request 页面的 Conversation(对话)传达他们想要的更改。
项目维护者要求对 Pull Request 进行一些更改。
可以看到源仓库的所有者进行了评论,并要求我们将 commit 结合在一起。
我越来越喜欢 Git 命令行工具和 GitHub 界面的一个原因在于,它们会提供下一步操作的建议,这非常有帮助。在评论区底部附近,GitHub 有一个建议,告诉我们如何添加更多的 commit;我们需要将它们添加到同一个分支并推送到我的分支:
Add more commits by pushing to the include-richards-destinations branch on richardkalehoff/course-collaboration-travel-plan .
在提交 Pull Request 时,记住你是要其他开发者将 你的 代码更改添加到他们的项目。如果他们要求你对你的 Pull Request 做一些小的(甚至是大的!)更改,这并不代表他们拒绝你的工作!只是他们希望以特定方式将你的代码添加到他们的项目中。
你应按照 CONTRIBUTING.md 文件列出项目维护者所需的所有信息,所以请遵照其中的信息。但有时项目维护者可能会要求你做一些其他事情。
那你该怎么做呢?如果你希望你的 Pull Request 被接受,那就执行要求的更改!记住,GitHub 中的选项卡称为"Conversation(对话)"选项卡。所以随时与项目维护者进行沟通,以明确他们希望你做什么。
你也应感谢他们花时间查看你的 Pull Request。大多数开源项目的开发者都是在免费做这些事情,所以记住:
- 善待他人 - 项目维护者只是像你一样的普通人
- 耐心 - 他们会尽快回复
Lam 要求我将我的更改结合在一起,之后她才会合并我的 Pull Request。将 commit结合在一起是一个称为压制的过程。我们来看看怎么做!
小结
就像看起来那么简单,主动发起 commit 请求主要在于沟通!
如果项目维护者要求更改 Pull Request,则:
- 在你的 Pull Request 所基于的本地仓库的同一分支上添加一些必要 commit
- 将该分支推送到 你的 源仓库 fork 副本
这些 commit 最终将显示在 Pull Request 页面上。